-
Notifications
You must be signed in to change notification settings - Fork 933
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added a heading in the client/operations.go file that was missing #11914
Added a heading in the client/operations.go file that was missing #11914
Conversation
Hi @ibilalkayy thanks for this. It looks like the documentation and dco checks are failing. |
@tomponline It's my first time sending the PRs here and I don't know how to pass these checks. What is lacking? |
The DCO one says here; https://github.com/lxc/lxd/actions/runs/5441251213/jobs/9895028225?pr=11914
|
@tomponline Although I visited the contribution guide but I was not able to understand where to sign off these details. Can you guide me? Thank you! |
please can you rebase to fix the documentation failure. |
Hi @ibilalkayy in this case as you just have one commit you can run If you run |
Signed-off-by: Bilal Khan <[email protected]>
@markylaing and @tomponline I added the signed-off details in the commit message and pushed it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Hey there,
There was documentation missing in one function in the client/operations.go file. I added 2 lines of text there to make them understandable.
Thank you!